[PW_SID:1063756] [v2,1/4] station: check return of handshake_state_set_pmksa#491
[PW_SID:1063756] [v2,1/4] station: check return of handshake_state_set_pmksa#491IWDTestBot wants to merge 14 commits intoworkflowfrom
Conversation
This is taken care of by the individual cache items and if none exist, tar fails.
If this fails num_pmkids and pmkids would get set, but to an uninitialized buffer. This would then fail to build the handshake object later when copying the PMKID.
This makes adding roaming tests easier if they don't strictly conform to the existing test structure. By adding connect/roam helpers future tests will have a bit more control on whats being tested.
This test exposes a bug where if a PMKSA already exists for a BSS that we are roaming to IWD crashes due to a missing check in station.c
The existing check was only looking at s->expiration and if it was zero or non-zero. If a PMKSA exists for a BSS the expiration will be non-zero, but that shouldn't preclude us from setting that into the handshake. Instead check if the PMKSA is actually expired, and only then return false.
|
Fetch PR Prep - Setup ELL Make Distcheck Build - Configure Make Check Make Check w/Valgrind Incremental Build with patches |
|
Fetch PR GitLint Prep - Setup ELL Make Distcheck Build - Configure Make Check Make Check w/Valgrind Incremental Build with patches Autotest Runner Output: Clang Build |
github-actions
bot
force-pushed
the
workflow
branch
3 times, most recently
from
7d4e7ec to
c723eef
Compare
2 participants
If this fails num_pmkids and pmkids would get set, but to an
uninitialized buffer. This would then fail to build the handshake
object later when copying the PMKID.
src/station.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)